Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

deploy the docs app from a PR to gh-pages #1595

Merged
merged 1 commit into from
Jul 24, 2024
Merged

Conversation

balzss
Copy link
Contributor

@balzss balzss commented Jul 22, 2024

INSTUI-4139

this pr creates 2 new actions:

  • one to build and deploy the docs app from a PR to gh-pages
  • and another to build and deploy the docs app on master push to gh-pages

(note that this doesn't replace instructure.design and only creates a parallel copy with the url https://instructure.github.io/instructure-ui/; a full replacement needs some additional things like the legacy docs and should be done in a separate ticket)

this pr also removes a part of the pr validation action (which runs the unit tests) which builds and deploys the docs to netlify

@balzss balzss self-assigned this Jul 22, 2024
Copy link

github-actions bot commented Jul 22, 2024

PR Preview Action v1.4.7
Preview removed because the pull request was closed.
2024-07-24 08:44 UTC

@balzss balzss changed the title test gh pages preview deploy the docs app from a PR to gh-pages Jul 22, 2024
Copy link
Collaborator

@matyasf matyasf left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nice work!

in the future we could exclude some paths that dont affect the docs build, e.g. test folders, Storybook folders, regression test https://docs.github.com/en/actions/using-workflows/workflow-syntax-for-github-actions#example-excluding-paths

@balzss balzss merged commit 4a5e679 into master Jul 24, 2024
9 checks passed
@balzss balzss deleted the test/gh-pages-preview branch July 24, 2024 08:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants